-
Notifications
You must be signed in to change notification settings - Fork 54
Hotfix/1.10.1 - Enable claiming in prod #2395
Conversation
* remove unused audio. * Rename and remove .asd file
* Improve responsiveness of header elements. * Fix breakpoints. * Reset max-width
|
* Using hardcoded gas values for gchain * Using Blockscout's gas oracle on gchain instead of hardcoded values * Adding 20% extra margin for gas estimation * Sorting the imports a little bit Co-authored-by: Leandro <[email protected]>
I retested changes for
And these changes LGTM there. |
Should be updated now |
Weird things are happening in this PR: I stop seeing investment options for an account when I switch to another network, then return back. |
* Get deployment time per network * Set cache
Fixed now. Also, I was not able to run tests for All the rest changes LGTM. |
* Show banner if the user has claimed already * Update claimed banner message and color * Change one const name
* Update vcow blog links based on chain id * Updated the blog root link const name * Updated the links hook name and const
* Showing claimed vCOW balance amount in summary when already claimed * Updating body message when user already claimed * Update button text when already claimed * Added isClaimed to `ClaimCommonTypes` * Show network name on already claimed banner * Do not show inner body msg when already claimed * Show connect button on already claimed if disconnected * Fix `@src` import Co-authored-by: Leandro <[email protected]>
* Updated state to store count of classified claims other than a boolean * Updated the updater to store the count of classified claims * Updated claims on other network banner to check according to claim counts * Fixed issues with claim on multiple networks - Updated state to add new dimension to claim info: account Otherwise when checking a different account the state would be preserved and would lead to bad data - Refactored Updater to fix dependency on obj causing it to being stuck in a re-render loop - Fixed logic that compared different types using `===` instead of `==` - Fixed logic that was causing claim to be displayed when there were no claims * Refactoring variable Co-authored-by: Leandro <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good.
Needed for the release:
- vCOW contract address for all networks
- Double check blog posts are correct
Awesome! Big day tomorrow 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
I have found a new UI issue in the PR #2415, but it might be addresses later.
* Use gip-13 data * Update contract addresses
Summary
Changelog
git log --oneline origin/hotfix/1.10.1...origin/master --no-decorate
60cfffe 2382/already claimed (#2394)
1ab7b5b Removed Ethereum word from popup text msg (#2412)
c7ba6d4 Show claimed context when already claimed (#2410)
56311fb Update vcow blog links based on chain id (#2408)
66ed3cb Show banner if the user has claimed already (#2393)
d65d7f0 Get deployment time per network (#2407)
8e3ae44 Update contract
c770626 Updated gas estimation for gchain (#2391)
0e0ebae Improve responsiveness of header elements. (#2376)
72d923e Add alternative claim success sound. (#2384)
0cab018 remove unused audio. (#2348)
5586fe9 Fix token display in dark/light mode for transactions. (#2386)
6e9d521 Enable claiming on prod and ens
Todo